Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

style: improve UserControls design #1324

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

United600
Copy link
Contributor

@United600 United600 commented Mar 16, 2024

Improves the look and visual states of the DismissButtonStyle (renamed to TransparentButtonStyle) and the respective UserControls that make use of it.

PR Type

  • Other... Please describe:

Find and Replace

  • Improve padding
  • Improve glyphs
  • Improve High Contrast
  • Improve ToggleReplaceModeButton touch target
  • Add ToggleReplaceModeButton animation
  • Add SelectionIndicator animation
FindAndReplace.mp4

GoTo

  • Improve padding
  • Improve glyphs
  • Improve HighContrast
GoTo.mp4

SideBySideDiffViewer

  • Improve brushes
  • Improve DismissButton size
SidebySideDiff.mp4

Other information

Adds some design attributes and better XAML formatting.

update dismiss button style and controls

improve spacing

improve brushes

improve glyphs

improve high contrast theme

add design-time attributes

format xaml
@United600 United600 force-pushed the controls branch 3 times, most recently from 87f1f67 to 75b963b Compare May 25, 2024 22:41
@United600 United600 changed the title style: improvements to a couple of UserControls style: improve UserControls design Jul 22, 2024
add ToggleReplaceModeButton animation
@0x7c13
Copy link
Owner

0x7c13 commented Nov 11, 2024

@United600 I have played around with latest WinUI3/WinAppSdk and decided to adopt the design and app model once its get matured. So far, there are still lots of issues presented(I have spent 30+ hours building an app using WinAppSDK so I know what I am speaking. NativeAOT support is also just realized by most of the parties. Guess we still need to wait for some time before taking any major actions. FYI: Your changes will be obsolete once we move to settings card controls etc when adopting WinUI3.

@United600
Copy link
Contributor Author

United600 commented Nov 11, 2024

That's sad to hear. Personally, I would have opted for WinUI 2 first and stuck with UWP in the meantime.
Sure some of the changes here would be obsolete, eg. Expander, however they are applicable to a future migration.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants